Skip to content

fix(deps): unify TinyAgents 2.1 source#121

Merged
senamakel merged 3 commits into
tinyhumansai:mainfrom
senamakel:fix/tinyagents-2-1-single-source
Jul 23, 2026
Merged

fix(deps): unify TinyAgents 2.1 source#121
senamakel merged 3 commits into
tinyhumansai:mainfrom
senamakel:fix/tinyagents-2-1-single-source

Conversation

@senamakel

@senamakel senamakel commented Jul 22, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Summary

  • declare TinyAgents 2.1 from crates.io so downstream workspaces can apply one canonical patch
  • keep standalone TinyCortex development on its nested pinned submodule via a local crates.io patch
  • bump the nested development pin and lockfile to TinyAgents v2.1.0

This prevents OpenHuman from resolving TinyCortex against a second TinyAgents path package, which would split the shared trait universe.

Validation

  • cargo fmt --all -- --check
  • ulimit -n 4096; cargo test --manifest-path Cargo.toml --all-features -- --test-threads=2 (1384 unit + 18 integration/doc tests passed; 2 ignored)

Summary by CodeRabbit

  • Chores
    • Updated standalone development configuration to use the repository’s vendored tinyagents dependency.
    • Host applications can continue selecting their own canonical dependency version.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 535a7809-4b9a-4610-a2b0-02c7dee31cfc

📥 Commits

Reviewing files that changed from the base of the PR and between f2908bf and aee435a.

📒 Files selected for processing (1)
  • Cargo.toml
📝 Walkthrough

Walkthrough

Cargo.toml adds a [patch.crates-io] override directing tinyagents to vendor/tinyagents during standalone development, with comments describing behavior when consumed by a host.

Changes

tinyagents development override

Layer / File(s) Summary
Vendored dependency override
Cargo.toml		 Adds a crates.io patch mapping tinyagents to vendor/tinyagents and documents host patch behavior.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

I’m a rabbit with a Cargo tune,
Patching tinyagents by the moon.
From vendor burrow, paths align,
Standalone builds now hop just fine.
Host crates choose their own design—
Squeak! The manifests intertwine.

Comment @coderabbitai help to get the list of available commands.

chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jul 22, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 12caf4479a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Cargo.toml
@senamakel
senamakel merged commit 5f46593 into tinyhumansai:main Jul 23, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments
@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@senamakel